Skip to content

[SYCL][Graph][Test] Add native recording test that instantiates the graph multiple times - #23614

Merged
sarnex merged 2 commits into
intel:syclfrom
adamfidel:matt/multi_exec_graph_native_recording
Oct 28, 2026
Merged

sarnex merged 2 commits into
intel:syclfrom
adamfidel:matt/multi_exec_graph_native_recording

Conversation

@mmichel11

Copy link
Copy Markdown
Contributor

It is valid to instantiate a graph multiple times. Executions of the same executable graph are generally linearized by the backend, even across different queues. Multiple instantiations enable concurrent execution as each executable graph instance is separate.

@mmichel11
mmichel11 marked this pull request as ready for review October 23, 2026 21:29
@mmichel11
mmichel11 requested a review from a team as a code owner October 23, 2026 21:29
@mmichel11
mmichel11 requested a review from adamfidel October 23, 2026 21:29

@adamfidel adamfidel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mmichel11
mmichel11 marked this pull request as draft October 28, 2026 15:46
@mmichel11
mmichel11 force-pushed the matt/multi_exec_graph_native_recording branch from 8cc6817 to affcbb0 Compare October 28, 2026 15:52
@mmichel11
mmichel11 marked this pull request as ready for review October 28, 2026 18:27
@mmichel11

Copy link
Copy Markdown
Contributor Author

I rebased and made the change to skip windows leak checking with this new test.

@mmichel11

Copy link
Copy Markdown
Contributor Author

@intel/llvm-gatekeepers Please consider merging.

@sarnex
sarnex merged commit 98c9ec4 into intel:sycl Oct 28, 2026
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants